layout: Move window_group visibility to the layout

This ensures that windows are hidden in the screen shield and in gdm.

https://bugzilla.gnome.org/show_bug.cgi?id=695747

diff --git a/js/ui/layout.js b/js/ui/layout.js
index 32b65e603be93306314490004d8d2c497e32ffdd..c38655e69a4832a420c4be70e6429d2388f43886 100644 (file)
--- a/js/ui/layout.js
+++ b/js/ui/layout.js
@@ -833,6 +833,11 @@ const LayoutManager = new Lang.Class({
     },
 
     _updateVisibility: function() {
+        let windowsVisible = Main.sessionMode.hasWindows && !this._inOverview;
+
+        global.window_group.visible = windowsVisible;
+        global.top_window_group.visible = windowsVisible;
+
         for (let i = 0; i < this._trackedActors.length; i++) {
             let actorData = this._trackedActors[i], visible;
             if (!actorData.trackFullscreen)
@@ -840,7 +845,7 @@ const LayoutManager = new Lang.Class({
             if (!actorData.isToplevel)
                 continue;
 
-            if (this._inOverview || !Main.sessionMode.hasWindows)
+            if (!windowsVisible)
                 visible = true;
             else if (this.findMonitorForActor(actorData.actor).inFullscreen)
                 visible = false;

diff --git a/js/ui/overview.js b/js/ui/overview.js
index 33cc478abe55ab4c46077a85b16ddee951fc888d..a6ed7fd7110960a1fbe6822d91583b769604833b 100644 (file)
--- a/js/ui/overview.js
+++ b/js/ui/overview.js
@@ -540,8 +540,6 @@ const Overview = new Lang.Class({
         //
         // Disable unredirection while in the overview
         Meta.disable_unredirect_for_screen(global.screen);
-        global.window_group.hide();
-        global.top_window_group.hide();
         this._stack.show();
         this._backgroundGroup.show();
         this._viewSelector.show();
@@ -688,9 +686,6 @@ const Overview = new Lang.Class({
         // Re-enable unredirection
         Meta.enable_unredirect_for_screen(global.screen);
 
-        global.window_group.show();
-        global.top_window_group.show();
-
         this._viewSelector.hide();
         this._desktopFade.hide();
         this._backgroundGroup.hide();